Skip to content
This repository was archived by the owner on Mar 4, 2020. It is now read-only.

feat(chat message): add author and timestamp props#242

Merged
mnajdova merged 9 commits intomasterfrom
feat/chat-author-and-timestamp
Sep 19, 2018
Merged

feat(chat message): add author and timestamp props#242
mnajdova merged 9 commits intomasterfrom
feat/chat-author-and-timestamp

Conversation

@bmdalex
Copy link
Collaborator

@bmdalex bmdalex commented Sep 17, 2018 •
edited
Loading

Chat.Message: author and timestamp

This PR introduces author and timestamp props for Chat.Message subcomponent

TODO

  • Conformance test
  • Minimal doc site example
  • Stardust base theme
  • Teams Light theme
  • Teams Dark theme
  • Teams Contrast theme
  • Confirm RTL usage
  • W3 accessibility check
  • Stardust accessibility check
  • Update glossary props table
  • Update the CHANGELOG.md

author

Rendered only in "their" messages (mine=false).

screen shot 2018-09-17 at 20 23 51

<Chat.Message author="John Doe" content="Hello there"  />

renders

<li class="ui-layout ui-chat__message">
  <div class="ui-layout__main">
    <div class="ui-layout">
      <div class="ui-layout__start">
        <span class="ui-text a">John Doe</span>
      </div>
      <div class="ui-layout__main">Hello there</div>
    </div>
  </div>
</li>

timestamp

screen shot 2018-09-17 at 20 23 42

<Chat.Message timestamp="Yesterday, 10:15 PM" content="Hello there"  />

renders

<li class="ui-layout ui-chat__message">
  <div class="ui-layout__main">
    <div class="ui-layout">
      <div class="ui-layout__start">
        <span class="ui-text a">Yesterday, 10:15 PM</span>
      </div>
      <div class="ui-layout__main">Hello there</div>
    </div>
  </div>
</li>

Loading
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@dvdzkwsk dvdzkwsk Awaiting requested review from dvdzkwsk

@levithomason levithomason Awaiting requested review from levithomason levithomason is a code owner

@miroslavstastny miroslavstastny Awaiting requested review from miroslavstastny miroslavstastny is a code owner

2 more reviewers

@mnajdova mnajdova mnajdova left review comments

@kuzhelov kuzhelov kuzhelov approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

ready for merge

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@bmdalex @mnajdova @kuzhelov